Skip to content

Conversation

@pratlucas
Copy link
Collaborator

When building the embedded toolchain, we currently disable assertions
for libcxxabi, but assertions for libunwind are still enabled. This
causes some of the logging functionality from libunwind to be included
in the build, which in turn imposes a dependency on IO functionality
that might not be available on baremetal environemnts.

This patch disables assertions for libunwind using the
-DLIBUNWIND_ENABLE_ASSERTIONS=OFF option, removing the issue.

When building the embedded toolchain, we currently disable assertions
for libcxxabi, but assertions for libunwind are still enabled. This
causes some of the logging functionality from libunwind to be included
in the build, which in turn imposes a dependency on IO functionality
that might not be available on baremetal environemnts.

This patch disables assertions for libunwind using the
`-DLIBUNWIND_ENABLE_ASSERTIONS=OFF` option, removing the issue.
Copy link
Collaborator

@ostannard ostannard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pratlucas pratlucas merged commit f960467 into ARM-software:main Oct 15, 2024
1 check passed
@pratlucas pratlucas deleted the libunwind_asserts branch October 15, 2024 09:27
voltur01 added a commit to voltur01/LLVM-embedded-toolchain-for-Arm that referenced this pull request Dec 19, 2024
Disable libunwind assertions to follow the main branch PR ARM-software#534 and resolve mismatch between libcxxabi and libunwind.
voltur01 added a commit that referenced this pull request Dec 19, 2024
Disable libunwind assertions to follow the main branch PR
[#534](#534)
and resolve mismatch between libcxxabi and libunwind.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants